✅ Deploy Preview for nebari-docs ready!

Name	Link
🔨 Latest commit	d1e9cd6
🔍 Latest deploy log	https://app.netlify.com/sites/nebari-docs/deploys/66eb194f35e7a90008e11b71
😎 Deploy Preview	https://deploy-preview-503--nebari-docs.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Can I get some help debugging the failure in the CI build?

The error is The engine "node" is incompatible with this module. Expected version ">=18.0". Got "16.20.2" but I've specified node>18 here. It builds fine on my machine so I think it must be something to do with the CI runner but I dont know enough about npm/yarn to sort it out.

Checked Netlify per @krassowski suggestions, and we do configure the Node version there. Updated to use 18.x now:

Re-built the site, and seems to be working on Netlify now. The preview above also works.

I am not sure why the GitHub CI is still red and not updating. @kcpevey I'll push an empty commit to this PR to re-trigger CI. Please let me know if you'd like me to refrain from pushing up to your branch in the future. :)

Please let me know if you'd like me to refrain from pushing up to your branch in the future.

I'll take all the help I can get - I always welcome collaboration :)

Thank you so much for looking into this @pavithraes ! And thanks for the assistance on finding the solution @krassowski !

This is now ready for review.

One thing that I do need resolved is the icon admonition (discussed in the description). If we're fine to remove, then its done. If it needs to stay, then I'll need help re-implementing it in a way that docusaurus v3 likes.

I removed the icon admonition as I think its no longer necessary. This is ready for review.

nice! Thanks @kcpevey this will likely be the last in the order of merges after your #502 and #476

I'll have to do conflict resolution again whenever we are close to merging.

Hi @kcpevey as #502 what's the status on this PR, is it ready for review?

@viniciusdc Now that blocking PR has been merged, I'll resolve conflicts and get things working again. I'll ping you when its ready for final review.

@viniciusdc this is ready for a final look!

@kcpevey I've noticed from the current built version of this PR, that some of the admonition colors and codeblocks styles are not showing up correctly, is this intended?

before:

current PR build:

I've noticed from the current built version of this PR, that some of the admonition colors and codeblocks styles are not showing up correctly, is this intended?

Hmmm. I'll have to look more closely. I did make a change to admonitions that may be having an effect here.

@viniciusdc The styling that was previously working for the admonitions is no longer being used. All of the admonitions now are using the css rules that start with alert (e.g. even for note). So the current rules are not applied.

There is also a comment in admonitions.scss about alerts. I played with this a bit, but the behavior seems to have changed and/or I dont have the skillz needed to sort it out.

So at this point, everything in admonitions.scss is being ignored and we are just getting the default admonition styling. I'm not sure if there is anyone available to spend the time to sort this out right now.

Thats okay thank you for having a look into it @kcpevey -- wonder if this might be due to docssaurus-sass plug-in

wonder if this might be due to docssaurus-sass plug-in

Its possible?

We brought up the plan to open an issue to resolve the admonition styling in a separate issue/PR and go ahead and merge this one since its so big. No one disagreed, so @viniciusdc I think we're good to go ahead and merge?